Skip to content

Conversation

@logaretm
Copy link
Member

@logaretm logaretm commented Feb 6, 2026

Implements HTTP server instrumentation for both h3 and srvx by listening to their tracing channel events.

⚠️ Note that both of these channels were renamed here and here, so we will hold off until those are out with Nitro's next version.

Closes #18123


This PR is part of a stack:

@linear
Copy link

linear bot commented Feb 6, 2026

@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2026

Codecov Results 📊


Generated by Codecov Action

@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2026

Codecov Results 📊

25 passed | ⏭️ 5 skipped | Total: 30 | Pass Rate: 83.33% | Execution Time: 13.58s

All tests are passing successfully.


Generated by Codecov Action

@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2026

size-limit report 📦

Path Size % Change Change
@sentry/browser 25.56 kB added added
@sentry/browser - with treeshaking flags 24.08 kB added added
@sentry/browser (incl. Tracing) 42.36 kB added added
@sentry/browser (incl. Tracing, Profiling) 47.03 kB added added
@sentry/browser (incl. Tracing, Replay) 81.18 kB added added
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 70.8 kB added added
@sentry/browser (incl. Tracing, Replay with Canvas) 85.87 kB added added
@sentry/browser (incl. Tracing, Replay, Feedback) 98.03 kB added added
@sentry/browser (incl. Feedback) 42.29 kB added added
@sentry/browser (incl. sendFeedback) 30.23 kB added added
@sentry/browser (incl. FeedbackAsync) 35.22 kB added added
@sentry/browser (incl. Metrics) 26.74 kB added added
@sentry/browser (incl. Logs) 26.88 kB added added
@sentry/browser (incl. Metrics & Logs) 27.56 kB added added
@sentry/react 27.33 kB added added
@sentry/react (incl. Tracing) 44.72 kB added added
@sentry/vue 30.01 kB added added
@sentry/vue (incl. Tracing) 44.21 kB added added
@sentry/svelte 25.58 kB added added
CDN Bundle 28.11 kB added added
CDN Bundle (incl. Tracing) 43.2 kB added added
CDN Bundle (incl. Logs, Metrics) 28.95 kB added added
CDN Bundle (incl. Tracing, Logs, Metrics) 44.03 kB added added
CDN Bundle (incl. Replay, Logs, Metrics) 68.02 kB added added
CDN Bundle (incl. Tracing, Replay) 80.07 kB added added
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 80.94 kB added added
CDN Bundle (incl. Tracing, Replay, Feedback) 85.5 kB added added
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 86.4 kB added added
CDN Bundle - uncompressed 82.22 kB added added
CDN Bundle (incl. Tracing) - uncompressed 127.93 kB added added
CDN Bundle (incl. Logs, Metrics) - uncompressed 85.05 kB added added
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 130.76 kB added added
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 208.71 kB added added
CDN Bundle (incl. Tracing, Replay) - uncompressed 244.81 kB added added
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 247.63 kB added added
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 257.61 kB added added
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 260.42 kB added added
@sentry/nextjs (client) 47.04 kB added added
@sentry/sveltekit (client) 42.81 kB added added
@sentry/node-core 52.17 kB added added
@sentry/node 166.52 kB added added
@sentry/node - without tracing 93.96 kB added added
@sentry/aws-serverless 109.45 kB added added

@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2026

node-overhead report 🧳

Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.

Scenario Requests/s % of Baseline Prev. Requests/s Change %
GET Baseline 9,178 - - added
GET With Sentry 1,699 19% - added
GET With Sentry (error only) 6,068 66% - added
POST Baseline 1,172 - - added
POST With Sentry 575 49% - added
POST With Sentry (error only) 1,038 89% - added
MYSQL Baseline 3,219 - - added
MYSQL With Sentry 448 14% - added
MYSQL With Sentry (error only) 2,622 81% - added

@logaretm logaretm force-pushed the awad/js-1128-nitro-use-tracing-channels-for-srvx-and-h3 branch 2 times, most recently from 23a8b35 to 07ad5a1 Compare February 9, 2026 20:05
@logaretm logaretm force-pushed the awad/create-nitro-sdk-metadata branch from 81b555e to 161862d Compare February 10, 2026 19:31
@logaretm logaretm force-pushed the awad/js-1128-nitro-use-tracing-channels-for-srvx-and-h3 branch from 6f1a78f to c3fef9e Compare February 10, 2026 19:34
@logaretm logaretm force-pushed the awad/js-1128-nitro-use-tracing-channels-for-srvx-and-h3 branch from c3fef9e to 6799854 Compare February 10, 2026 20:23
import { setServerTimingHeaders } from '../hooks/setServerTimingHeaders';

export default definePlugin(nitroApp => {
// FIXME: Nitro hooks are not typed it seems

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This has been fixed in nitrojs/nitro#4014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants